gdk: Rename gdk_event_get_history() to get_motion_history()

It only applies to GDK_MOTION_NOTIFY events, so this is a more descriptive
name. Also add Since tag in docs, and GDK_AVAILABLE_IN_3_94.

build: Fix the non-pkg-config Vulkan detection

Not all the variables were renamed.

build: Create input method module cache directory

Before running gtk-query-immodules, the cache output directory may not
actually exist. Make sure it does before we try to write into it.

https://bugzilla.gnome.org/show_bug.cgi?id=793182
Signed-off-by: Daniel Stone <daniels@collabora.com>

build: Use pkg-config to find Vulkan

The standard Vulkan SDK ships with a pkg-config file, like a modern
library should.

We should fall back to finding the library and header only for platforms
where pkg-config is not really a thing.

Based on a patch by: Daniel Stone <daniels@collabora.com>

https://bugzilla.gnome.org/show_bug.cgi?id=793181

Drop the last mentions of motion hint events

The GDK_POINTER_MOTION_HINT_MASK enumeration value is gone, but we're
still keeping around the "is_hint" field in GdkEventMotion, even though
every backend sets it to `false` — except for the core X11 device
manager.

Don't put gdk types into a gsk header

This is a leftover from when GskTexture was moved
to gdk.

Drop the mir backend

Mir is in the process of switching over to the Wayland protocol,
so we don't need to maintain a separate backend, going forward.

Update Polish translation

docs: Remove some leftovers

There were still references to recent chooser and tool palette
types in the docs.

Drop threads from the docs

Drop gtk_widget_show_now

This function does bad things, and is not an
API that we want to take into GTK+ 4.

treeview: Stop using gtk_widget_show_now

If this is a problem, it needs to be handled explicitly.

gtk testsuite: Stop using gtk_widget_show_now

If this is a problem, it needs to be handled explicitly.

a11y testsuite: Stop using gtk_widget_show_now

On non-windows, this doesn't do anything anyway.

searchbar: reorganize

Avoid the ugly priv->tool_box==NULL check in ::add (and ::remove) by
just not using template xml for this small class. Also, make sure the
GtkBin child is properly set and implement remove to also properly
remove it. Remove the manual widget margins and add some CSS for it.
Also switch to simply using a GtkCenterBox.

searchbar: Properly remove child widgets

Since the GtkSearchBar is not the direct parent, we need to remove them
properly from priv->box_center.

searchbar: Remove unused widgets

both box_left and box_right are unused.

gdk: fix a format string warning

glarea: Fix precondition check

and the build.

roundedrect: Fix coordinate mixup in contains_point

This together with the optimizations in GtkSnapshot was causing circular
buttons to become squared when partially scrolled out of view.

glarea: Simplify make_current

We only use the widget pointer here once.

gl renderer: Use a gl texture in render_texture

Get rid of the manual y flipping and cairo business. This keeps the gl
unit tests working at least.

Update POTFILES.in

rendernode: Don't alloca() 8MB

This happens when deserializing testcases and it really confuses
valgrind into thinking we're longjmp()ing.

And deserializing rendernodes is slow anyway, so who cares about a few
more malloc()s.

rbtree: Privatize header

According to meson it's the only private header that doesn't conform to
the *private.h naming.

Fix that.

Allow binding GdkContentFormatsBuilder

GdkContentFormatsBuilder is currently not introspectable, as it does not
have a GType. We can turn it into a boxed type, but we need to implement
memory management for it.

The current gdk_content_formats_builder_free() function returns a newly
constructed value, so we cannot use it as a GBoxedFreeFunc; additionally
copying a GdkContentFormatsBuilder contents would make it a bit odd, as
you could get multiple identical GdkContentFormats out of the copies.

A simple approach is to model the GdkContentFormatsBuilder API to follow
the GBytes one: use reference counting for memory management, and have
a function to release a reference, return a GdkContentFormats, and reset
the GdkContentFormatsBuilder state.

For language bindings, we can provide a get_formats() function that
returns the GdkContentFormats instance and resets the builder instance,
leaving the reference count untouched.

For C convenience we can keep gdk_content_formats_builder_free(), and
make it a wrapper around gdk_content_formats_builder_get_formats(), with
the guarantee that it'll free the builder instance regardless of its
current reference count.

https://bugzilla.gnome.org/show_bug.cgi?id=793097
https://blogs.gnome.org/otte/2018/02/03/builders/

gl renderer: Clear resources after render_texture

This makes multiple consecutive render_texture calls on the same
renderer work.

notebook: Handle touch events for tabs

No need to check for the event type here.

widget: Remove realized checks from _translate_coordinates

As discussed.

widget: Don't call snapshot vfunc if width or height are 0

The result won't be visible anyway. This also prevents problems with
widgets that create some resource the size of the widget, like
GtkGLArea. It also keeps us from snapshotting revealers with
size 0.

iconbrowser: setup image dnd for image6

Otherwise the we fail to set the drag source icon name later on.

Remove all gdk_threads_* entry points

Now that we don't use them anywhere, it's time for them to go.

https://bugzilla.gnome.org/show_bug.cgi?id=793124

Drop the Big GDK Lock

GDK has a lock to mark critical sections inside the backends.
Additionally, code that would re-enter into the GTK main loop was
supposed to hold the lock.

Back in the Good Old Days™ this was guaranteed to kind of work only on
the X11 backend, and would cause a neat explosion on any other GDK
backend.

During GTK+ 3.x we deprecated the API to enter and leave the critical
sections, and now we can remove all the internal uses of the lock, since
external API that uses GTK+ 4.x won't be able to hold the GDK lock.

https://bugzilla.gnome.org/show_bug.cgi?id=793124

Replace gdk_threads_add_timeout* with g_timeout_add()

The main GDK thread lock is not portable and deprecated.

The only reason why gdk_threads_add_timeout() and
gdk_threads_add_timeout_full() exist is to allow invoking a callback
with the GDK lock held, in case 3rd party libraries still use the
deprecated gdk_threads_enter()/gdk_threads_leave() API.

Since we're removing the GDK lock, and we're releasing a new major API,
such code cannot exist any more; this means we can use the GLib API for
installing timeout callbacks.

https://bugzilla.gnome.org/show_bug.cgi?id=793124

Replace gdk_threads_add_idle* with g_idle_add()

The main GDK thread lock is not portable and deprecated.

The only reason why gdk_threads_add_idle() and
gdk_threads_add_idle_full() exist is to allow invoking a callback with
the GDK lock held, in case 3rd party libraries still use the deprecated
gdk_threads_enter()/gdk_threads_leave() API.

Since we're removing the GDK lock, and we're releasing a new major API,
such code cannot exist any more; this means we can use the GLib API for
installing idle callbacks.

https://bugzilla.gnome.org/show_bug.cgi?id=793124

toolpalette: Remove

The widget was basically unmaintained since GTK 3.0 and the only known
user was Glade.

gtk: Remove GtkRecentChooser

It's not used any more, recent files are only used by the filtchooser
and that one uses GtkRecentManager directly.

tests: Unrealize renderer

Renderers must be unrealized these days.

texture: Add sanity checks to constructors

width, height and GL texture ID may not be 0, so return_if_fail() if
they are.

texture: Make the texture id a guint

Texture IDs are unsigned, so treat them like that.

Rename GtkActionMuxer's private header

Follow the naming policy for private headers.

Rename GtkActionHelper private header

Follow the naming scheme for private headers.

Split list of sources

We need to have two lists: one, with the list of sources that need to be
introspected; and one with the list of sources that contain only private
symbols.

This reduces the amount of source files that the introspection scanner
needs to traverse, and thus the build time.

docs: Annotate gdk_keymap_get_display()

Avoid a warning from the introspection scanner.

Remove unnecessary gtk-doc stanza

The places sidebar is not a publicly documented API, so there's no need
to use a gtk-doc stanza — especially if the syntax of the stanza is
wrong.

Remove leading underscore from private symbols

There's no need to do that any more, as only explicitly annotated
symbols are exported.

Rename gtkiconcachevalidator.h

Follow the same convention for private headers as newer parts of GTK.

Rename gtkiconcache.h

The gtkiconcache.h header is private, so rename it to follow the naming
scheme for private headers.

Replace boilerplate in GtkIcon

Be a good GObject citizen and use G_DECLARE_FINAL_TYPE instead of
writing the usual GObject boilerplate.

iconview: Fix a problem with the previous change

After commit ffef28a7e8d5ffef7de6a3baccb30b0021b6b0ff,
gtk-icon-browser was spewing critical warnings when
changing sections. Avoid that by respecting the return
value of gtk_tree_model_get_iter.

dnd: Remove unused member variable

dnd: Move GdkDragProtocol to X11

It's not needed in the generic implementation, so don't have it there.

dnd: Remove 2 vfuncs that aren't needed

They're only used inside the X11 backend, and the backend can just call
its own function.

icon browser: Remove usage of stock-size property

Does not exist anymore.

Revert "menu: Simplify popup_at_rect"

This reverts commit a29306cb1e739afaa4a7b31d48c4bec328cfef9e.

range: Always queue an allocate if the adjustment changed

The slider gets its new size in size-allocate, so we have to do this
even if the range has no origin.

wayland: Add support for BTN_STYLUS3

BTN_STYLUS3 is defined by the Linux 4.15 kernel and is sent when the
third button on a stylus is pressed. At the moment, only Wacom's "Pro
Pen 3D" has three stylus buttons. Pressing this button triggers a button
8 event to be sent under X11, so we use the same mapping here.

https://bugzilla.gnome.org/show_bug.cgi?id=790033

Add AGPL3-only licence to GtkAboutDialog

https://bugzilla.gnome.org/show_bug.cgi?id=792793

gl: Add test case for clipped cross fade nodes

gl renderer: Reset clip when drawing offscreen

Just like we reset the scissor test and the viewport, we also can't use
the current clip anymore in this case.

GskRoundedRect: Fix GSK_ROUNDED_RECT_INIT macro

GskRoundedRect does not have a .rect member, it's 'bounds'. Also,
properly initialize all fields, i.e. all the corners.

gl renderer: Fix hidpi cairo node rendering

We can't just unconditionally create a larger texture here, since the
incoming cairo surface might have a device scale that doesn't fit our
scale_factor. Instead, look up the surface device scale and use that.

gl renderer: Make width/height relation more obvious

In add_offscreen_ops, we use the max_x/min_x values for the texture and
viewport width, and the max_y/min_y for the texture and viewport height.

inspector: Actually call the pick() vfunc when picking

Otherwise, overridden pick vfuncs don't work and we e.g. pick widgets
scrolled away in a viewport.

entry: Remove unused struct members

menu: Simplify popup_at_rect

No need to keep a separate current_event variable around, we don't use
it anyway.

gdk: Stop referring to ::key-press/release-event

These signals are going away, don't mention them in
the docs.

entry completion: Stop using key-press/release-event

We can use the existing ::event handler for this.

gsk: Add test case for last commit

gl renderer: Fix outset shadow outline transform

This fixes hidpi blurred outset shadows

Enforce UTF-8 encoding when opening C files

We have a couple of Python 3.x scripts that parse C files, and since C
does not have any encoding, we need to force one ourselves, to avoid the
case when we're running the build in a non-UTF-8 locale.

https://bugzilla.gnome.org/show_bug.cgi?id=792497

Widget: Don’t call reset() on NULL EventController

GtkGesture is a GtkEventController. gtk_event_controller_dispose() calls
_gtk_widget_remove_controller(). That NULLs the pointer-to-Controller in
our EventControllerData but does not delete said ECData from our GList.

Subsequently, if that same Widget gets unparent()ed, that method calls
unset_state_flags(), which leads to doing reset_controllers() if we are
insensitive. Now, unlike most most other loops over the GList of ECData,
reset_controllers() does not skip nodes whose pointer-to-Controller is
NULL. So, we call gtk_event_controller_reset(NULL) and get a CRITICAL.

This surfaced in a gtkmm program. The Gesture is destroyed before the
Widget. The Widget then gets dispose()d, which calls unparent()… boom.
I didn’t find an MCVE yet but would hope this logic is correct anyway:

The simplest fix is to make the loop in gtk_widget_reset_controllers()
skip GList nodes with a NULL Controller pointer, like most other such
loops, so we avoid passing the NULL to gtk_event_controller_reset().

In other, live cases, _gtk_widget_run_controllers() loops over the GList
and removes/frees nodes having NULL Controllers, so that should suffice.
But this clearly was not getting a chance to happen in the failing case.

https://bugzilla.gnome.org/show_bug.cgi?id=792624

Update Esperanto translation

(cherry picked from commit 814c55d473c6a7ab9f930421aad5612a351ba68d)

GtkGestureMultiPress: check event state before emiting released signal

Fix bug 771986 "Inconsistent 'row-activated' signal emission before \
drag'n'drop, 'activate-on-single-click'=TRUE, 'reorderable'=TRUE"

widget: Only initialize bounds rect if necessary

We only use the graphene_rect_t version of the offset_clip for the
fallback cairo nodes.

vulkan: Fix release builds

combobox: Remove some unneeded includes

Set GDK_WINDOW_STATE_TILED if any edge is tiled

This state flag is used in several places in GTK+, for example to
ignore RESIZE_INC hints if tiled. Setting it is also necessary for
backwards compatibility with applications that changed their behaviour
when tiled, such as GNOME Terminal and its MATE fork.

Signed-off-by: Simon McVittie <smcv@debian.org>
https://bugzilla.gnome.org/show_bug.cgi?id=789357

display-x11: Unset tiled state if _GTK_EDGE_CONSTRAINTS is supported

Commit c415bef5de introduced support for the new _GTK_EDGE_CONSTRAINTS
atom. If the compositor supports that atom, however, we were always
setting the tiled state, even if no actual tiling information is
available, where the correct action is to completely remove any traces
of the tiled state.

Fix that by correctly removing the tiled state when compositor supports
_GTK_EDGE_CONSTRAINTS Xatom.

https://bugzilla.gnome.org/show_bug.cgi?id=788516

entrycompletion: Realize toplevel before attempting a grab

Otherwise, gtk_widget_get_window returns NULL and we can't successfully
perform a grab via the later gdk_set_grab call. This fixes the entry
completion in the file chooser not working.

tests: Remove some unneeded gtk_widget_show calls

Widgets are visible by default now.

iconview: Fix updates with filter models

Filter models rely on views taking a ref on every node
they care about. GtkIconView was not doing that. Amazingly,
this has never shown up in a bug so far, until I spotted
the fallout in gnome-font-viewer.

Add a test for ::row-changed vs node refs

Test that filter models propagate ::row-changed if there is
an external reference on the node, and not otherwise. This
is showing up in buggy icon view behaviour, where the icon
view is not redrawing if the content changes in a model that
is below a filter model.

gtkwindow: Disconnect GdkSeat::device-removed callback on finalize

Otherwise dangling callbacks may lead to crashes.

gl renderer: Fix shadow node child offset

We shouldn't apply the shadow offset when drawing the child offscreen,
instead apply it afterwards when we draw it to the current render
target.

testsuite: Add another gl renderer test case

linkbutton: Only set widget cursor once

inspector: Allow picking insensitive widgets

The default gtk_widget_contains/gtk_widget_pick don't consider
insensitive widgets.

csswidgetnode: Avoid some unnecessary work

The later code would just ref both styles and not do anything else.

widget-factory: Fix some layout issues

We have to explicitly set some of these to their default values so
expand-set is TRUE and the boxes stop propagating their expansion state
up the hierarchy.

textview: Remove some stray debugging output

Leftover from 7046463b880c79b227594a2b35a6c625119d286e

Simplify all ui files again

Now even simpler!

builder-tool: Try harder to find types

If g_type_from_name fails, try gtk_builder_get_type_from_name
before giving up. This fixes cases like GtkPrinterOptionWidget.

window: Ignore gtk_window_close calls from close-request handlers

widget: Always assign values to out parameter in translate_coordinates

safe guard against people using the returned coordinate values without
checking the return value of gtk_widget_translate_coordinates.

widget: Initialize a local variable

We will later pass this variable on to some other functions, so be safe
against them using the value.

aboutdialog: Only close on delete events

Clicking on the Credits button should not close the dialog.

fontchooswerwidget: Center spinbutton next to scales vertically

gl renderer: Render GL textures upside down

These come from an FBO so we need to flip the texture coords on the y
axis, just like we do everywhere else.

Revert "inspector: Stop adding event controllers to the object tree"

This reverts commit 8f3cb2658a79d20912b6f2ec96bc7e6bbdaeb067.

This didn't quite work out, since being in the object tree is
a prerequisite for showing up in the property editor.